chore: update dependencies and add Claude Code support#16
Merged
Conversation
- Add CLAUDE.md with project guidance for Claude Code - Update all npm dependencies to latest compatible versions - Fix TypeScript type error in lib/base64.ts - Fix ESLint module reassignment in lib/compiler.ts - Add ESLint configuration (.eslintrc.json) - Update README.md with clearer setup instructions - Rebuild public/index.json with updated examples - Fix 6 moderate security vulnerabilities in dependencies
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #16 +/- ##
=======================================
Coverage ? 36.32%
=======================================
Files ? 33
Lines ? 501
Branches ? 29
=======================================
Hits ? 182
Misses ? 307
Partials ? 12 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
- Upgrade Node.js from 18 to 20 to support vite@7.1.1 engine requirements - Add --with-deps flag to install required system dependencies for Playwright
- Replace __dirname with import.meta.dirname for ES modules - Update browser config to use new instances format instead of deprecated name field
- Exclude Monaco Editor and WASM-dependent tests that require browser environment - Keep working unit tests (lib/utils.test.ts) that pass in jsdom - This allows CI to pass while maintaining test coverage for core utilities
There was a problem hiding this comment.
Pull Request Overview
This pull request updates dependencies and adds Claude Code support to improve the development experience and fix security vulnerabilities. The changes focus on modernizing the Node.js toolchain while maintaining compatibility with the Ruby WASM-based WeakAuras DSL.
- Updated all npm dependencies to latest compatible versions, fixing 6 moderate security vulnerabilities
- Added Claude Code configuration documentation for better AI-assisted development
- Fixed TypeScript and ESLint issues that emerged from dependency updates
Reviewed Changes
Copilot reviewed 9 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updated npm dependencies to latest versions, added ESLint configuration |
| vitest.config.mts | Updated to use import.meta.dirname, removed browser testing, excluded test files |
| lib/compiler.ts | Fixed ESLint module reassignment warning by renaming variable |
| lib/base64.ts | Fixed TypeScript type error by narrowing Decoder interface parameter |
| .eslintrc.json | Added basic Next.js ESLint configuration |
| CLAUDE.md | Added comprehensive documentation for Claude Code integration |
| README.md | Updated with clearer setup and development instructions |
| .github/workflows/test.yml | Updated Node.js version and Playwright installation |
| public/index.json | Added new example files for various WoW classes/specs |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
lib/base64.tslib/compiler.tsTest plan
npm install- clean installnpm run dev- dev server startsnpm run build- production build succeedsnpm run lint- linting passes